Skip to content

[tests] tighten dependency testing.#13332

Merged
sayakpaul merged 7 commits intomainfrom
revisit-deps-tests
Apr 10, 2026
Merged

[tests] tighten dependency testing.#13332
sayakpaul merged 7 commits intomainfrom
revisit-deps-tests

Conversation

@sayakpaul
Copy link
Copy Markdown
Member

@sayakpaul sayakpaul commented Mar 25, 2026

What does this PR do?

We should be able to catch stuff like #13321.

Additionally, I ported the existing dependency test suite to use pytest.

@HuggingFaceDocBuilderDev
Copy link
Copy Markdown

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

- main
paths:
- "src/diffusers/**.py"
- "tests/**.py"
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I initially did it to trigger the tests in this PR. Will remove before merging.

@sayakpaul sayakpaul requested a review from DN6 March 25, 2026 08:54
Copy link
Copy Markdown
Collaborator

@DN6 DN6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks 👍🏽

@github-actions github-actions bot added tests pipelines CI size/M PR with diff < 200 LOC labels Apr 10, 2026
@sayakpaul
Copy link
Copy Markdown
Member Author

Failing tests are unrelated.

@sayakpaul sayakpaul merged commit 896fec3 into main Apr 10, 2026
15 of 19 checks passed
@sayakpaul sayakpaul deleted the revisit-deps-tests branch April 10, 2026 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI pipelines size/M PR with diff < 200 LOC tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants